Skip to content

Enable Qodana scans #456

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
rob93c merged 2 commits into from
Jan 13, 2026
Merged

Enable Qodana scans #456

rob93c merged 2 commits into from
Jan 13, 2026

Conversation

@rob93c
Copy link
Member

@rob93c rob93c commented Jan 12, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

Chores

  • Updated code quality analysis tool to the latest version (2025.3) and switched to the community edition.
  • Refined continuous integration configuration with optimised caching and baseline comparison settings.
  • Enabled unconditional code quality analysis in the automated build pipeline.
  • Enhanced reporting parameters for streamlined feedback during development.

✏️ Tip: You can customize this high-level summary in your review settings.

@rob93c rob93c self-assigned this Jan 12, 2026
@rob93c rob93c added the cicd The change affects CI/CD flows label Jan 12, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 12, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

Updates Qodana code analysis tooling configuration across two files: removes the conditional job guard, upgrades the action to v2025.3, adds explicit workflow parameters, and switches to the community edition image.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/unit-test.yml		 Removed conditional gating (if: clause) enabling the job to run unconditionally; upgraded Qodana action from v2025.2 to v2025.3; added explicit configuration arguments (--baseline,qodana.baseline.json, use-caches: false, post-pr-comment: false, pr-mode: false)
Qodana Configuration
qodana.yaml		 Updated Docker image references to community edition: jetbrains/qodana-jvm → jetbrains/qodana-jvm-community (version 2025.3)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

dependencies

Poem

🐰 A swift upgrade hops our way,
Community tools now lead the day,
No gates to block the testing flow,
Version 3's a leap—let analysis go!
Code quality at speeds anew, ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Enable Qodana scans' directly reflects the main objective of the pull request, which enables Qodana code quality scanning by removing conditional gatekeeping, updating to the community edition, and configuring explicit action parameters.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dfcded1 and 4f65b31.

📒 Files selected for processing (2)
  • .github/workflows/unit-test.yml
  • qodana.yaml
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings 
Learnt from: rob93c
Repo: Stickerifier/Stickerify PR: 298
File: qodana.yaml:6-6
Timestamp: 2024-11-02T11:09:02.322Z
Learning: The `jetbrains/qodana-jvm:2024.2` linter has been released and is available on DockerHub as of November 2024, and can be safely used in `qodana.yaml` files.
📚 Learning: 2024-11-02T11:09:02.322Z 
Learnt from: rob93c
Repo: Stickerifier/Stickerify PR: 298
File: qodana.yaml:6-6
Timestamp: 2024-11-02T11:09:02.322Z
Learning: The `jetbrains/qodana-jvm:2024.2` linter has been released and is available on DockerHub as of November 2024, and can be safely used in `qodana.yaml` files.

Applied to files:

  • qodana.yaml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
  • GitHub Check: Execute unit tests (macos-latest)
  • GitHub Check: Execute unit tests (ubuntu-latest)
  • GitHub Check: Execute unit tests (windows-latest)
  • GitHub Check: Build and upload Docker image
  • GitHub Check: Execute unit tests (ubuntu-latest)
  • GitHub Check: Execute unit tests (windows-latest)
  • GitHub Check: Execute unit tests (macos-latest)
🔇 Additional comments (2)
qodana.yaml (1)

6-7: LGTM! Switch to community edition with version bump comment.

The change to qodana-jvm-community is sensible for enabling Qodana scans without requiring a paid licence. The image jetbrains/qodana-jvm-community:2025.3 is available on DockerHub and actively maintained. The bump comment follows a standard pattern for automated version tracking.

.github/workflows/unit-test.yml (1)

80-87: LGTM! Qodana action upgrade with explicit configuration.

The update to v2025.3 aligns with the linter version in qodana.yaml (jetbrains/qodana-jvm-community:2025.3). The explicit arguments provide clear, reproducible behaviour:

  • --baseline,qodana.baseline.json — uses a baseline for suppressing known issues
  • use-caches: false — ensures clean analysis each run
  • pr-mode: false — performs full analysis rather than incremental

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rob93c rob93c requested a review from MartelliEnrico January 13, 2026 07:19
MartelliEnrico
MartelliEnrico approved these changes Jan 13, 2026
View reviewed changes
Copy link
Contributor

@MartelliEnrico MartelliEnrico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rob93c rob93c merged commit 2106453 into main Jan 13, 2026
14 of 17 checks passed
@rob93c rob93c deleted the enable-qodana-scans branch January 13, 2026 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MartelliEnrico MartelliEnrico MartelliEnrico approved these changes

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

@rob93c rob93c

Labels

cicd The change affects CI/CD flows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rob93c @MartelliEnrico